Skip to content

Do not use service_name for endpoint selection. Fix: #2074 - #2176

Open
micafer wants to merge 4 commits into
trunkfrom
fix_openstak_endpoints
Open

Do not use service_name for endpoint selection. Fix: #2074#2176
micafer wants to merge 4 commits into
trunkfrom
fix_openstak_endpoints

Conversation

@micafer

@micafer micafer commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Do not use service_name for endpoint selection

Description

Select service catalog endpoints by service type without filtering by the default service name. The service name is only used as a filter when explicitly provided via ex_force_service_name.

Status

  • done, ready for review

Checklist (tick everything that applies)

  • Code linting (required, can be done after the PR checks)
  • Documentation
  • Tests
  • ICLA (required for bigger changes)

@codecov-commenter

codecov-commenter commented Jul 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.58%. Comparing base (c26989b) to head (108c9c1).

Additional details and impacted files
@@            Coverage Diff             @@
##            trunk    #2176      +/-   ##
==========================================
+ Coverage   83.57%   83.58%   +0.01%     
==========================================
  Files         352      352              
  Lines       81788    81832      +44     
  Branches     8765     8767       +2     
==========================================
+ Hits        68353    68399      +46     
+ Misses      10562    10561       -1     
+ Partials     2873     2872       -1     
Files with missing lines Coverage Δ
libcloud/common/openstack.py 86.55% <100.00%> (+1.23%) ⬆️
libcloud/test/common/test_openstack.py 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@micafer
micafer requested a review from tisonkun July 31, 2026 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants